css tests: Update progressbar test results

This was a documented change that has not been reflected
in the tests yet.

css tests: Update entry test results

Just visibility changes here.

css nodes: Update the combobox test results

Some changes in the details here; we removed an old widget name.

css tests: Update the GtkSwitch node tests

The node hierarchy for GtkSwitch has changed.

switch: Update css docs to reflect current reality

This should have been done when the hierarchy was changed.

infobar: Remove unused defines

widgetfocus: Use the same function to compare widgets

... in horizontal/vertical direction.

Implement tab/up-down/left-right focus sorting for widgets

basically do what GtkContainer already did.

Widget: Shuffle focus code around

tests: Add widget focus test case

Shows an example of a widget that directly inherits from GtkWidget and
has multiple focusable children.

statusbar: Remove margins in ui file

Widgets themselves shouldn't have margins assigned.

widget: Allow focusing widgets with non-container parent

Especially if said parent also has can-focus set to FALSE, which is a
special-case we had before for GtkContainer instances.

label: Fix focus implementation

Don't return TRUE if we didn't actually focus anything.

window: Simplify _set_focus

window: Remove _gtk_window_set_is_toplevel

All GtkWindow instances are toplevels.

window: Fold update_has_focus in to set_is_active

We're now only ever setting has_focus to the same value we set is_active
to, so also remove that field from the private struct.

window: Remove has-toplevel-focus property

Since embedded toplevels don't exist anymore, :has-toplevel-focus is
equivalent to :active.

window: Remove check_resize special case for embedded toplevels

Those don't exist anymore.

window: Remove gtk_quark_embedded

Unused.

widget: Emit ::focus regardless of container-ness and can-focus

container: Remove focus_child pointer

Use GtkWidget's instead.

widget: Also reset focus on non-container widgets

Make gtk_container_set_focus_child private

Remove gtk_container_get_focus_child

widget: Save pointer to focus child

Do the same thing GtkContainer does.

cssimageicontheme: Cache size + texture

glrenderer: Reuse the GArray for render items

glrenderer: Add render items for child nodes

glrenderer: Handle color nodes

glrenderer: Store uniform locations per program

widget: Fix checks in gtk_widget_snapshot()

If the widget isn't drawable anyway, just return;

If the widget needs an allocate, print a warning, since it indicates a
problem in the widget workflow (e.g. forgot to size_allocate a child
widget).

This maches the previous checks in gtk_widget_draw (with the same
problems).

filechooserbutton: Hide the combobox by default

recentchooserwidget: Inherit from GtkWidget

appchooserwidget: Inherit from GtkWidget

fontchooserwidget: Inherit from GtkWidget

fontchooserwidget: Remove unused defines

filechooserwidget: Inherit from GtkWidget

Remove the Raleigh theme

It's been broken for ages.

widget: Add a GtkBuildable::add_child implementation

Just call gtk_widget_set_parent in there.

progressbar: Remove unused defines

flowbox: don’t select when rubberbanding over nothing

When rubberbanding over an empty area, everything is selected on gesture
end, which is a bit counter-intuitive.

https://bugzilla.gnome.org/show_bug.cgi?id=780735

osx: fix build

https://bugzilla.gnome.org/show_bug.cgi?id=734946

gdkwindow: subsurface in gdk_window_get_parent()

When the GtkWidget hierarchy does not match the GdkWindow hierarchy, the
GtkWidget code may find a common ancestor that cannot be found while
traversing the GdkWindow tree using gdk_window_get_parent().

This happens with for example on Wayland, a GtkPopover has another
GtkPopover as parent, in this case, the GdkWindow parent is the root
window, whereas the GtkWidget parent is the other GtkPopover.

That confuses the gtk_widget_translate_coordinates() logic which will
bail out in this case and won't return the translated coordinates.

Make gdk_window_get_effective_parent() aware of subsurfaces and use the
transient_for which represents the actual parent (whereas the parent
might be pointing to the root window).

Bugzilla: https://bugzilla.gnome.org/show_bug.cgi?id=774148

Work around a wayland crash

For some reason, we are seeing damage being NULL here.
While that should never be the case, crashing on it is
unkind and makes the Wayland experience unusable.

Document GTK_INSPECTOR_RENDERER

Also add GTK_INSPECTOR_DISPLAY to the docs at the same time.

gtk_widget_intersect: fix annotations

https://bugzilla.gnome.org/show_bug.cgi?id=773228

gskrendernodeimpl: Fix documentation argument type

GSK: Fix a bunch of warnings from the GIR scanner

arising from conflicting argument names, missing annotations, etc.

TextView: Be const-correct when passing a pointer

The TextIter is passed by pointer for efficiency. We neither need to
modify it, nor should we leave it possible to accidentally do so. So,
it should be passed as a pointer-to-const.

TextView: Get line direction in more efficient way

We do not need to go through the heavyweight process of constructing a
TextLineDisplay just to get the direction out of it, when we can simply
use TextIter API to get the text and then get its direction using Pango.

Adapted from a patch by Mehdi Sadeghi for GtkSourceView:

https://bugzilla.gnome.org/show_bug.cgi?id=779081#c20

TextBuffer: Add missing transfer full annotations

Update Polish translation

printjob: Clarify array ownership in gtk_print_job_set_page_ranges()

Add a documentation annotation saying that set_page_ranges transfers
ownership of the GtkPageRange array.
Add a g_free() call to fix a memory leak when set_page_ranges is
used repeatedly.

https://bugzilla.gnome.org/show_bug.cgi?id=780234

Avoid more compiler warnings

Avoid unused variable warnings

Avoid redefining GNU_SOURCE

Gives a compiler warning with the wrong flags.

Avoid some compiler warnings

printing: Don't hang in gtk_enumerate_printers()

Stop enumerating printers when all backends were removed.

https://bugzilla.gnome.org/show_bug.cgi?id=686838

Update Russian translation

(cherry picked from commit a7f3fd12d5c4e866a1d8e05360a7052a756ee8bd)

mir: always emit a resize when creating windows

Update German translation

glcontext: Fix buffer age comparison

The old code had a dead >=3 branch, causing flickering.

gdkdragsource: Ensure button >= 1

So the left shift later doesn't overflow.

widget: Prepend style classes to list when parsing

Since the later gtk_style_context_add_class doesn't care about the order
of the style classes, we can as well just prepend style classes to the
list and avoid the squared behavior when appending to a linked list.

builderparser: Remove pointless TagType struct

Update Indonesian translation

Scrollbar: grammar fix for previous commit

Scrollbar: Tweak intro docs a bit

Explain where the adjustment comes from, clarify some of the wording
about how its fields influence the scrollbar, and also note that the
steppers may not be present, since they aren’t in our default themes.

ScrolledWindow: Don’t check if child is Scrollable

If the child added is not a Scrollable, it gets wrapped in a ViewPort –
which is. So it is impossible to end up with a non-Scrollable child.

Just check we have /any/ child where needed, which is semantically nicer

https://bugzilla.gnome.org/show_bug.cgi?id=778853

ScrolledWindow: Streamline/clarify the intro docs

 • intro: Clarify that external policy and/or adjustments can be used.
 • add(): Don’t waffle on about having to add a ViewPort since we handle
          that transparently for the user, so they can add() any widget.
 • Adjustment stuff: most of this was repeating the docs for Scrollbar,
                     so just refer the user to that. Also, mention how
                     policies NEVER and EXTERNAL interact with all this.

https://bugzilla.gnome.org/show_bug.cgi?id=778853

Tooltip: Avoid redundant variable initialisations

has_tooltip_widget was assigned twice in immediate succession.

return_value is not used anywhere else in this function since commit
14a864c8b55dfe92f8957499f12a3f9303188a12 and does not need a default
value anymore, so move it to the inner scope and don't init to NULL.

GdkWaylandWindow: Force window titles to valid utf8

Use the new g_utf8_make_valid in case the given title is not valid utf8.
Also bump the glib requirement to 2.51.5.

label: Properly calculate half the height

Since we compare it to a double anyway, might as well divide by 2.0.

tooltip: Remove unused assignment

hide_tooltip  gets overriden in any case 2 lines down, and return_value
isn't used later in that function. The second assignment was introduced
in ef1da5f6c2450fc5f7c7de4a17114cc7507a41ad, directly below the first
assignment.

css-editor: Remove unused if statement

The static local is set to NULL and never set to anything else, so we
never use it anyway.

window: Remove pointless ternary operator

we're already checking for orientation == GTK_ORIENTATION_HORIZONTAL
right before this line, so just pass GTK_ORIENTATION_VERTICAL.

box: Make sure center_req is initialized

In case we have an invisible center widget, we never initialize it, but
later still use it.

progressbar: Remove useless if statement

Both the if and the else block contained exactly the same code.

gsk: Take scale factor into account when setting up the gl scissor

gl renderer: Fix hidpi fallback rendering

When allocating the surface, take scale into consideration.

Update Friulian translation

notebook: Remove dead assignment

GtkRecentManager: Remove superfluous local variable

GdkRenderer: Fix a compiler warning

GdkX11DeviceManager: Fix debugging output

The 2 values added in 3.22 were missing from the source_names array.

GdkMonitor: Use 1 as scale fallback value

The same value we use in gtk_widget_get_scale_factor.

css-overview: Fix inconsistent British English use

vulkan: fix compilation

entrycompletion: Remove unnecessary NULL check

completion really shouldn't be NULL at this point, especially since it
gets dereferenced higher up in the function.

calendar: Fix else branch indentations

GdkDrawingContext: Add missing break in switch statement

css-overview: Fix+Explain color expr number ranges

Last try, promise. They don’t all use 0 to 1. We should probably explain
the effects too. Hopefully this manages that while not being too verbose

Avoid a critical warning in the filechooser portal

This was showing up when cancelling a portal file chooser
in recipes.

css-overview: Elaborate how color expressions work

docs/css-overview: Fix color functions’ arg orders

shade/alpha/mix() take colour(s) and a number that is the ratio by which
to transform them. It was written here that these shall be passed in the
order (number, colour). That was wrong: they must be passed in the order
(colour[s], number) to work, and for the Inspector not to flag an error.

Update Latvian translation

docs: Remove mentions of gtk_init_with_args().

Because it has been removed: See commit 7e8eb3ddcd29ce9eecb14a56dbfeb0caf64dc772

headerbar: Fix size requisition variable mixup

Happened when porting to gtk_widget_measure.

GtkShaderBuilder Fix copy/paste error in precondition

gtk_shader_builder_add_define should check both define_name and
define_value for not-NULL and not-empty, but the second precondition
check checks define_name again for not-empty-ness.

Allow custom renderer for the inspector

If you set GTK_INSPECTOR_RENDERER to the same type of
values that GSK_RENDERER takes this can change the renderer
used for the inspector. This is useful if you're debugging
one renderer and don't want to affect the inspector.